Skip to content

Conversation

@millord
Copy link
Collaborator

@millord millord commented Dec 26, 2025

This PR addresses #5712

  • Addresses the issue in full
  • Addresses only certain aspects of the issue

Description

The purpose of this ticket is to change the default text for the Deposit helper text field from " or one month's rent may be higher for lower credit scores" to Deposit will not exceed one month's rent

How Can This Be Tested/Reviewed?

1- navigate to the partners site
2-Click on Add listing
3-select any jurisdiction
3-Navigate to the Additional fees section
4- Verify the Deposit helper text has this copy: Deposit will not exceed one month's rent

image image

Provide instructions so we can review, including any needed configuration, and the test cases that need to be QAd.

Author Checklist:

  • Added QA notes to the issue with applicable URLs
  • Reviewed in a desktop view
  • Reviewed in a mobile view
  • Reviewed considering accessibility
  • Added tests covering the changes
  • Made corresponding changes to the documentation
  • Ran yarn generate:client and/or created a migration when required

Review Process:

  • Read and understand the issue
  • Ensure the author has added QA notes
  • Review the code itself from a style point of view
  • Pull the changes down locally and test that the acceptance criteria is met
  • Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

@millord millord changed the title updated copy for seeds files feat: Change default text for deposit helper text field Dec 26, 2025
@netlify
Copy link

netlify bot commented Dec 26, 2025

Deploy Preview for bloom-flagly canceled.

Name Link
🔨 Latest commit d2b620c
🔍 Latest deploy log https://app.netlify.com/projects/bloom-flagly/deploys/695d91c54fad2600089b1d96

@netlify
Copy link

netlify bot commented Dec 26, 2025

Deploy Preview for bloom-angelopolis canceled.

Name Link
🔨 Latest commit d2b620c
🔍 Latest deploy log https://app.netlify.com/projects/bloom-angelopolis/deploys/695d91c646c1ad0008a9f106

@netlify
Copy link

netlify bot commented Dec 26, 2025

Deploy Preview for partners-bloom-dev ready!

Name Link
🔨 Latest commit d2b620c
🔍 Latest deploy log https://app.netlify.com/projects/partners-bloom-dev/deploys/695d91c55bc6ff00084b96ef
😎 Deploy Preview https://deploy-preview-5727--partners-bloom-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@millord millord marked this pull request as ready for review December 26, 2025 17:47
@millord millord added the 1 review needed Requires 1 more review before ready to merge label Dec 26, 2025
@emilyjablonski
Copy link
Collaborator

emilyjablonski commented Jan 6, 2026

@millord Please fill out the PR template before tagging as ready for review! We're missing an issue number, a description, and the checklist

@emilyjablonski emilyjablonski removed the 1 review needed Requires 1 more review before ready to merge label Jan 6, 2026
@millord millord added the 1 review needed Requires 1 more review before ready to merge label Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 review needed Requires 1 more review before ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants